View Issue Details

IDProjectCategoryView StatusLast Update
0001737T99X171.00 SKB EagleSW Issuepublic2023-07-27 16:20
Reporter(ALTech) Younkwang Jung Assigned To(ALTech) Younkwang Jung Due Date2023-06-15 14:32
PriorityurgentSeveritys4-minorReproducibilityalways
Status closedResolutionfixed 
Summary0001737: [Smart3][ATV12] Issue of STB not being able to wake up after energy-saving mode setting
DescriptionHi Kerwin

https://jira.skbroadband.com/browse/FSTB12-117

After the Google settings set below, STB can enter energy-saving mode.
구글 설정(Google setting) -> 설정(Settings) -> 기기 환경설정(Device Perferences) -> 에너지 절약(Energy saver)

The problem is not waking up after entering energy-saving mode.
This issue is viewed as critical by SKB, so please check it quickly.

Thank you
YK.Jung
TagsNo tags attached.
Attach Tags

Users monitoring this issue

User List (ALTech) JunGyu Kim , (ALTech) SY Yoon

Activities

(SW) Jerry Lin

2023-06-12 19:08

developer   ~0013342

Hi YK,
Please help to discuss with SPtek.
Normal case (RCU Power Key to standby mode):
06-12 06:09:02.332 W/HdmiControlService( 3730): onStandby btv_hdmi_control_enabled : 1, Intent (sys.stb.on) = 0 <<=== sys.stb.on is 0

Failed case (energy-saving mode)
06-12 06:09:02.332 W/HdmiControlService( 3730): onStandby btv_hdmi_control_enabled : 1, Intent (sys.stb.on) = 1 <<=== sys.stb.on is 1

We add the following code in HdmiControlService.java.
diff --git a/services/core/java/com/android/server/hdmi/HdmiControlService.java b/services/core/java/com/android/server/hdmi/HdmiControlService.java
index 4f7b229f412b..a902dc46f367 100755
--- a/services/core/java/com/android/server/hdmi/HdmiControlService.java
+++ b/services/core/java/com/android/server/hdmi/HdmiControlService.java
@@ -3313,6 +3313,13 @@ public class HdmiControlService extends SystemService {
                        return;
                }
                // add SPTEK.MW : 2023.01.12 : [BPM-963] + E
+ if(!checkHdmiCecPackageForCts() && SystemProperties.get("sys.stb.on").equals("1")) {
+ Slog.d(TAG, "hdmi_standby SendBroadcast Intent (com.skbtv.tv.sleep) extra (hdmi)");
+ Intent intent = new Intent("com.skbtv.tv.sleep");
+ intent.putExtra("reason", "hdmi");
+ getContext().sendBroadcast(intent);
+ return;
+ }
         mWakeUpMessageReceived = false;
         assertRunOnServiceThread();
         setActiveness(HdmiCecActiveness.CEC_INACTIVE);


Jerry

(ALTech) Younkwang Jung

2023-06-13 11:03

developer   ~0013346

Hi Jerry

I will check the code you told me now.

And
as far as I know, in ATV10, this function was disabled by default
Please disable(없음) as well by default at ATV12 ( attached picture )
Was the ATV10 patch related to this not applied to ATV12?
And Even if you modify the default value, there is no problem with Google approval, is it correct?

please check it

Thank you
YK.Jung
image.png (45,054 bytes)   
image.png (45,054 bytes)   

(SW) Jerry Lin

2023-06-13 14:53

developer   ~0013351

Hi YK,
This commit is from ATV10. I will check it in ATV12.
If it is ok , i will commit it to ATV12.

====> project: device/google/atv
commit 9533e14886008f2c931af887c8aa09252d4f08e3
Author: dixon_chen <dixon_chen@ott-build.server>
Date: Wed May 8 08:54:19 2019 +0800

    Change the default value of Screen saver Put device to sleep

    From "After 3 hours of inactivity" , To "Never"

diff --git a/overlay/frameworks/base/packages/SettingsProvider/res/values/defaults.xml b/overlay/frameworks/base/packages/SettingsProvider/res/values/defaults.xml
index be6b533..91d8fb6 100644
--- a/overlay/frameworks/base/packages/SettingsProvider/res/values/defaults.xml
+++ b/overlay/frameworks/base/packages/SettingsProvider/res/values/defaults.xml
@@ -34,7 +34,7 @@
     <bool name="def_netstats_enabled">false</bool>

     <!-- ms until the system goes to sleep, 3 hours default for TV devices -->
- <integer name="def_sleep_timeout">10800000</integer>
+ <integer name="def_sleep_timeout">-1</integer>

     <!-- Default screen timeout set when setting screensaver. Currently (15 * 60 * 1000) = 15 min -->
     <integer name="def_screen_off_timeout">900000</integer>

Jerry

(ALTech) Younkwang Jung

2023-06-15 13:40

developer   ~0013388

Last edited: 2023-06-15 13:43

View 2 revisions

Hi Jerry.

As I checked it again, it seems to be a RCU(BT) issue

1) connect RCU(BT) to STB
2) STB enters energy-saver
3) tester can't wake up STB with the RCU(BT).
   ==> If tester use RCU(IR) here, STB can wake up.

This issue seems to be related to RCU(BT).
Please check it again

Additionally, there are SKB's requests as follows.
https://mantis.cnsbg.foxconn.com/vaas/view.php?id=1741

Thank you
YK.Jung

(SW) Jerry Lin

2023-06-16 10:45

developer   ~0013405

Hi YK,
Please help to check this commit.
We disable wake up STB with the RCU(BT) in bootloader.
And only support wake up STB with RCU(IR) in passive standby mode.

Jerry

jerry@010870-build57-workstation:~/project/Eagle/BFX_AT100_A12$ rp show be7efe17e4d25bfac54e392287c6c0d6ddb718d6
====> project: bootloader/uboot-repo/bl33/v2015
commit be7efe17e4d25bfac54e392287c6c0d6ddb718d6 (tag: BFX-AT100_5.3.4_S_DEV_2023-0103-144203, tag: BFX-AT100_5.3.4_S_DEV_2023-0103-143823, tag: BFX-AT100_5.3.4_S_DEV_2022-1228-143847, tag: BFX-AT100_5.3.4_S_DEV_2022-1228-143410, tag: BFX-AT100_5.3.4_S_DEV_2022-1222-143825, tag: BFX-AT100_5.3.4_S_DEV_2022-1222-143441)
Author: Bcan <bcan.cc.yeh@fii-foxconn.com>
Date: Thu Dec 15 14:56:15 2022 +0800

    Disable BT in bootloader

diff --git a/board/amlogic/configs/g12a_u212_v1.h b/board/amlogic/configs/g12a_u212_v1.h
index d9febe653a..ca6ae17f2c 100644
--- a/board/amlogic/configs/g12a_u212_v1.h
+++ b/board/amlogic/configs/g12a_u212_v1.h
@@ -33,9 +33,10 @@
 // Foxconn modified by JerryLin , disable CEC wakeup in bootloader , 2019/02/15
 //#define CONFIG_CEC_WAKEUP
 /*if use bt-wakeup,open it*/
-#define CONFIG_BT_WAKEUP
+// disable BT wakeup in bootloader
+//#define CONFIG_BT_WAKEUP
 /*if use uart rcu,open it*/
-#define CONFIG_BT_RCU_WAKEUP
+//#define CONFIG_BT_RCU_WAKEUP
 /* SMP Definitinos */
 #define CPU_RELEASE_ADDR secondary_boot_func

diff --git a/board/amlogic/g12a_u212_v1/firmware/scp_task/pwr_ctrl.c b/board/amlogic/g12a_u212_v1/firmware/scp_task/pwr_ctrl.c
index 9b832adc95..b45171c407 100644
--- a/board/amlogic/g12a_u212_v1/firmware/scp_task/pwr_ctrl.c
+++ b/board/amlogic/g12a_u212_v1/firmware/scp_task/pwr_ctrl.c
@@ -171,6 +171,7 @@ static unsigned int detect_key(unsigned int suspend_from)
                        exit_reason = RTC_WAKEUP;
                }

+ #ifdef CONFIG_BT_WAKEUP
                if (irq[IRQ_GPIO1] == IRQ_GPIO1_NUM) {
                        irq[IRQ_GPIO1] = 0xFFFFFFFF;
                        if (!(readl(PREG_PAD_GPIO2_I) & (0x01 << 18))
@@ -210,6 +211,7 @@ static unsigned int detect_key(unsigned int suspend_from)
                        #endif
                        }
                }
+ #endif

                if (irq[IRQ_ETH_PTM] == IRQ_ETH_PMT_NUM) {
                        irq[IRQ_ETH_PTM]= 0xFFFFFFFF;

(ALTech) Younkwang Jung

2023-06-19 16:49

developer   ~0013447

Hi Jerry

When the STB enters the energy-saver
I think the mode of RCU should be changed from BT mode to IR mode.
Then this issue will be solved

Thank you
YK.Jung

(ALTech) Younkwang Jung

2023-06-26 20:15

developer   ~0013504

Hi jerry

There will be ATV12 FW tagging tomorrow.
Please solve this issue quickly.

Thank you
YK.Jung

(SW) Jerry Lin

2023-06-26 20:43

developer   ~0013505

Hi YK,
Becuase it needs to comply with the low power specification, the BT wakeup of SWAN and SMART3 is prohibited.

Jerry

(ALTech) Younkwang Jung

2023-06-27 05:25

developer   ~0013506

Hi Jerry

There's a misunderstanding about what I'm talking about.

Of course, when STB enter low power mode, STB can't wakeup with BT.

The point of this issue is that when STB enters low power mode, RCU in BT mode should switch to IR mode.

Please let me know if I misunderstand.

Thank you
YK.Jung

(SW) Jerry Lin

2023-06-29 17:22

developer   ~0013532

Hi YK,

upload to
/release_by_fxn/temp_wfh/jerry/eagle/mantis_1741/ATV12_SD/20230629/usb_bfx-at100_V15.537.53.zip

Jerry

(ALTech) Younkwang Jung

2023-06-30 12:22

developer   ~0013534

Hi Jerry

There is no problem with the QA team test result.
Please commit it to UI537

And I created JIRA "BPM-20102"
https://jira.skbroadband.com/browse/BPM-20102

Please put JIRA number at commit message and commit
===================================
[BPM-20102] ... commit message
===================================

Also, please let me know the cause of this issue and how it was fixed.
I'll update that on JIRA

Thank you
YK.Jung

(ALTech) Younkwang Jung

2023-07-03 09:16

developer   ~0013537

Hi Jerry

Until when can you commit?
Please let me know when the commit is done

Thank you
YK.Jung

(SW) Jerry Lin

2023-07-03 10:06

developer   ~0013542

Hi YK,
commit done!

2023-07-03 09:59:37 | liang-chyi.lin@fii.. | cff3e70b3 | packages/apps/Bluetooth | [BPM-20102] Add a property check to change RCU moe from BT to IR in energy-saving mode [2/2]
2023-07-03 09:58:50 | liang-chyi.lin@fii.. | fb138d026ab0 | frameworks/base | [BPM-20102] Add a property check to change RCU moe from BT to IR in energy-saving mode [1/2]

Jerry

Issue History

Date Modified Username Field Change
2023-06-12 13:35 (ALTech) Younkwang Jung New Issue
2023-06-12 13:35 (ALTech) Younkwang Jung Status new => assigned
2023-06-12 13:35 (ALTech) Younkwang Jung Assigned To => (SW) Kerwin Chen
2023-06-12 13:36 (ALTech) Younkwang Jung Issue Monitored: (ALTech) SY Yoon
2023-06-12 13:36 (ALTech) Younkwang Jung Issue Monitored: (ALTech) JunGyu Kim
2023-06-12 15:13 (SW) Kerwin Chen Assigned To (SW) Kerwin Chen => (SW) Jerry Lin
2023-06-12 17:56 (ALTech) Younkwang Jung Severity s2-severe => s4-minor
2023-06-12 19:08 (SW) Jerry Lin Note Added: 0013342
2023-06-13 09:12 (SW) Jerry Lin Assigned To (SW) Jerry Lin => (ALTech) Younkwang Jung
2023-06-13 11:03 (ALTech) Younkwang Jung Note Added: 0013346
2023-06-13 11:03 (ALTech) Younkwang Jung File Added: image.png
2023-06-13 14:53 (SW) Jerry Lin Note Added: 0013351
2023-06-15 13:40 (ALTech) Younkwang Jung Note Added: 0013388
2023-06-15 13:43 (ALTech) Younkwang Jung Note Edited: 0013388 View Revisions
2023-06-16 10:45 (SW) Jerry Lin Note Added: 0013405
2023-06-19 16:49 (ALTech) Younkwang Jung Note Added: 0013447
2023-06-26 20:15 (ALTech) Younkwang Jung Note Added: 0013504
2023-06-26 20:43 (SW) Jerry Lin Note Added: 0013505
2023-06-27 05:25 (ALTech) Younkwang Jung Note Added: 0013506
2023-06-29 17:22 (SW) Jerry Lin Note Added: 0013532
2023-06-30 12:22 (ALTech) Younkwang Jung Note Added: 0013534
2023-07-03 09:16 (ALTech) Younkwang Jung Note Added: 0013537
2023-07-03 10:06 (SW) Jerry Lin Note Added: 0013542
2023-07-27 16:20 (ALTech) Younkwang Jung Status assigned => closed
2023-07-27 16:20 (ALTech) Younkwang Jung Resolution open => fixed